Skip to content

feat: Security fix and pre-commit fixes#906

Merged
vara-bonthu merged 2 commits into
mainfrom
security-fix
Sep 26, 2025
Merged

feat: Security fix and pre-commit fixes#906
vara-bonthu merged 2 commits into
mainfrom
security-fix

Conversation

@vara-bonthu

@vara-bonthu vara-bonthu commented Sep 26, 2025

Copy link
Copy Markdown
Collaborator

What does this PR do?

🛑 Please open an issue first to discuss any significant work and flesh out details/direction. When we triage the issues, we will add labels to the issue like "Enhancement", "Bug" which should indicate to you that this issue can be worked on and we are looking forward to your PR. We would hate for your time to be wasted.
Consult the CONTRIBUTING guide for submitting pull-requests.

  • Fixed security recommendations from code scanning alerts
  • Fixed pre-commit errors with terraform validate

Motivation

More

  • Yes, I have tested the PR using my local account setup (Provide any test evidence report under Additional Notes)
  • Mandatory for new blueprints. Yes, I have added a example to support my blueprint PR
  • Mandatory for new blueprints. Yes, I have updated the website/docs or website/blog section for this feature
  • Yes, I ran pre-commit run -a with this PR. Link for installing pre-commit locally

For Moderators

  • E2E Test successfully complete before merge?

Additional Notes

@vara-bonthu vara-bonthu requested review from alanty and nabuskey and removed request for nabuskey September 26, 2025 17:43
PyYAML==6.0.1
pyzmq==26.2.1
requests==2.32.3
requests>=2.32.3

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be set to a specific version because we want consistency

Signed-off-by: vara-bonthu <vara.bonthu@gmail.com>

@nabuskey nabuskey left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@vara-bonthu vara-bonthu merged commit 2f47684 into main Sep 26, 2025
28 checks passed
@vara-bonthu vara-bonthu deleted the security-fix branch September 26, 2025 19:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants